Skip to content

style(favorite): made the favorite button more visible #41

Jump to bottom
Merged
merged 1 commit into from
Sep 19, 2026
Merged

style(favorite): made the favorite button more visible #41

merged 1 commit into from
Sep 19, 2026
+56 −15

Conversation

ayushb
Copy link
Member

@ayushb ayushb commented Sep 18, 2026

The white heart emoji was almost invisible on the white card.

  • replaced the emoji with an inline svg heart so it renders the same on every OS
  • gave the button the same border and background as the nav buttons, red fill when favorite
  • updated the tests and snapshot

@ayushb
style(favorite): made the favorite button more visible
97c8539 
* Replaced the emoji with an inline svg heart
@ayushb ayushb requested a review from a team as a code owner September 18, 2026 23:20
thomhet
thomhet approved these changes Sep 18, 2026
View reviewed changes
Copy link
Member

@thomhet thomhet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ayushb ayushb merged commit 526b448 into main Sep 19, 2026
@ayushb ayushb deleted the feat/favorite-button-visibility branch September 19, 2026 08:47
Sign in to join this conversation on GitHub.
Reviewers

@thomhet thomhet thomhet approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ayushb @thomhet